Skip to content

fix: disable failing mutation tests#847

Merged
vbreuss merged 1 commit intomainfrom
topic/disable-mutation-tests
Nov 20, 2025
Merged

fix: disable failing mutation tests#847
vbreuss merged 1 commit intomainfrom
topic/disable-mutation-tests

Conversation

@vbreuss
Copy link
Copy Markdown
Member

@vbreuss vbreuss commented Nov 20, 2025

This PR temporarily disables mutation tests by adding a configuration flag and applying conditional execution logic. The change allows the build pipeline to skip mutation testing until stryker-mutator/stryker-net#3327 is resolved.

Key changes

  • Added a DisableMutationTests boolean flag set to true
  • Applied conditional execution to MutationTests and MutationTestDashboard targets using the new flag

@vbreuss
fix: disable failing mutation tests
9fab9d9 
This PR temporarily disables mutation tests by adding a configuration flag and applying conditional execution logic. The change allows the build pipeline to skip mutation testing until stryker-mutator/stryker-net#3327 is resolved.

### Key changes
- Added a `DisableMutationTests` boolean flag set to `true`
- Applied conditional execution to `MutationTests` and `MutationTestDashboard` targets using the new flag
@vbreuss vbreuss self-assigned this Nov 20, 2025
Copilot AI review requested due to automatic review settings November 20, 2025 15:21
@vbreuss vbreuss added the dependencies Pull requests that update a dependency file label Nov 20, 2025
Copilot AI reviewed Nov 20, 2025
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR temporarily disables mutation tests to work around a known issue in Stryker.NET (issue #3327) by introducing a configuration flag that conditionally skips mutation test execution.

Key changes:

  • Introduces a DisableMutationTests flag set to true to control mutation test execution
  • Applies conditional execution logic to all mutation test targets using the new flag

Comment thread Pipeline/Build.MutationTests.cs
@vbreuss vbreuss merged commit de27b02 into main Nov 20, 2025
8 of 9 checks passed
@vbreuss vbreuss deleted the topic/disable-mutation-tests branch November 20, 2025 15:22
github-actions Bot added a commit that referenced this pull request Nov 20, 2025
@github-actions
Update benchmark for de27b02: fix: disable failing mutation tests (#847…
ad9bbaa 
…) by Valentin Breuß
github-actions Bot added a commit that referenced this pull request Nov 20, 2025
@github-actions
Update benchmark for de27b02: fix: disable failing mutation tests (#847…
f2ec1b8 
…) by Valentin Breuß
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 20, 2025

🚀 Benchmark Results

Details 
BenchmarkDotNet v0.15.6, Linux Ubuntu 24.04.3 LTS (Noble Numbat)

AMD EPYC 7763 2.69GHz, 1 CPU, 4 logical and 2 physical cores

.NET SDK 10.0.100

[Host] : .NET 8.0.22 (8.0.22, 8.0.2225.52707), X64 RyuJIT x86-64-v3


Job=InProcess  Toolchain=InProcessEmitToolchain  IterationCount=15

LaunchCount=1  WarmupCount=10
Method Mean Error StdDev Gen0 Gen1 Allocated
Bool_aweXpect 276.6 ns 2.31 ns 2.05 ns 0.0424 - 712 B
Bool_FluentAssertions 251.9 ns 4.65 ns 4.35 ns 0.0567 - 952 B
Equivalency_aweXpect 307,948.8 ns 871.60 ns 680.48 ns 20.0195 0.4883 335452 B
Equivalency_FluentAssertions 2,700,299.7 ns 51,870.43 ns 48,519.63 ns 285.1563 46.8750 4804906 B
Int_GreaterThan_aweXpect 258.0 ns 1.96 ns 1.64 ns 0.0486 - 816 B
Int_GreaterThan_FluentAssertions 247.4 ns 2.16 ns 2.02 ns 0.0730 - 1224 B
ItemsCount_AtLeast_aweXpect 495.1 ns 2.63 ns 2.33 ns 0.0811 - 1368 B
ItemsCount_AtLeast_FluentAssertions 471.1 ns 5.39 ns 4.50 ns 0.1197 - 2008 B
String_aweXpect 468.8 ns 2.41 ns 2.26 ns 0.0677 - 1136 B
String_FluentAssertions 1,299.1 ns 11.61 ns 10.86 ns 0.2346 - 3944 B
StringArray_aweXpect 1,948.1 ns 20.64 ns 19.31 ns 0.1564 - 2632 B
StringArray_FluentAssertions 1,332.3 ns 7.45 ns 6.97 ns 0.2480 - 4152 B
StringArrayInAnyOrder_aweXpect 2,490.3 ns 12.87 ns 12.03 ns 0.1678 - 2824 B
StringArrayInAnyOrder_FluentAssertions 89,613.1 ns 419.28 ns 350.12 ns 3.4180 - 58598 B

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 21, 2025

This is addressed in release v2.29.0.

@github-actions github-actions Bot added the state: released The issue is released label Nov 21, 2025
This was referenced Nov 24, 2025
Closed
Closed
This was referenced Dec 1, 2025
Closed
Closed
This was referenced Jan 26, 2026
Closed
Open
@dependabot dependabot Bot mentioned this pull request Feb 9, 2026
Closed
This was referenced Mar 30, 2026
Merged
Closed
This was referenced May 2, 2026
Open
Closed
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

@vbreuss vbreuss

Labels

dependencies Pull requests that update a dependency file state: released The issue is released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vbreuss